-
Notifications
You must be signed in to change notification settings - Fork 14.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Airflow Standard Provider #41564
Airflow Standard Provider #41564
Conversation
d179424
to
f2f5fbd
Compare
daf5108
to
7b9296e
Compare
One more thing - at least for now, "standard" should be added to list of preinstalled providers in
|
We might change where it is added in Airflow 3, but for now it should be added here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we are missing few things...
- we need to deprecate the existed operators in core
- we are missing all the operators and sensors we have in core (currently it's only for time operators)
- I am not sure what about the
hooks
folder?
|
ee24a98
to
aa7fd34
Compare
Thanks @romsharon98 , will be adding rest of the operators and sensors to this. |
@eladkal me and romsharon discussed , i will be adding rest of the operators and sensors. |
Ah .. so no ... We should not add it to preinstalled just yet - until we release it (sorry @romsharon98 -> see the failure on openapi client). |
Then I propose this PR to remove the files from main that we migrated to the provider. |
Ah! I didn't see changes to |
Please correct me if i understood this, on v2-10-test deprecation updates required for all the changes we are doing on main? |
No - only those that are supposed to affect DAG authors. |
BTW. We will have to make the old "standard" providers still work in Airflow 3 I think so we will have to make the old operator imports work (likely with the mechanism mentioned by @ashb - @ashb ? Do you agree ? |
cad107f
to
78b8d2a
Compare
Interesting. The errors we see are very useful :).
|
revert the example dags to be under also not sure but should I change somewhere in the code to tell that this provider is mandatory and will auto installed? |
That's the preinstalled_providers - but it will only work after it's been released and it will likely change with Airflow 3 so for now - no need. |
…om python 2.8 and 2.9 competability
9760b46
to
27708ff
Compare
* add core.time provider * add source-date-epoch * change core to essentials * revert external task sensor location * add provider to airflow_providers_bug_report list * change new provider name to standard * change changelog * add integration * add standard provider to pre-install list * revert hatch_build * move examples back to airflow core * fixing mock * add title to commit * fix time.rst * change sensors example dags paths * revert example get_test_run addition * remove init * revert howto docs * fix docs * update standard provider airflow minimum version, remove standarad from python 2.8 and 2.9 competability * change provider as not-ready * add changelog * move how to docs to provider * fix docs images * fix import error * revert global constant * change time_sensors.py to time.py * revert min version to be 2.10 * remove providers from 2.9 and 2.8 * revert global constant * add operators guid to provider yaml
Extract all time operators and sensors from airflow core and remove them to
standard
providerMailing list thread: https://lists.apache.org/thread/2dmlqkcmyomm4q7rrovygs6bw655zx07
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rst
or{issue_number}.significant.rst
, in newsfragments.